Repository navigation
fix(skills): remove phantom slash commands from hermes-agent SKILL.md - #50608
liuhao1024 wants to merge 1 commit into
Conversation
- Remove `/skill <name>` (does not exist in command registry; only `/skills` exists) - Fix `/quit` alias: remove `/q` (it is an alias for `/queue`, not `/quit`) - Update troubleshooting section to reference `hermes -s name` instead of nonexistent `/skill` Fixes NousResearch#50605
teknium1
left a comment
There was a problem hiding this comment.
Thanks for identifying a real registry/documentation mismatch. Current main confirms the core correction: q belongs to /queue (hermes_cli/commands.py:110-111), /quit has only exit (hermes_cli/commands.py:253-254), and there is no built-in singular skill command (hermes_cli/commands.py:177-181).
Problems
- The patch updates only
skills/autonomous-ai-agents/hermes-agent/SKILL.md. The published English bundled-skill page still contains all three stale references atwebsite/docs/user-guide/skills/bundled/autonomous-ai-agents/autonomous-ai-agents-hermes-agent.md:307,:353, and:850. - The zh-Hans bundled-skill mirror has the same stale references at
website/i18n/zh-Hans/docusaurus-plugin-content-docs/current/user-guide/skills/bundled/autonomous-ai-agents/autonomous-ai-agents-hermes-agent.md:296,:342, and:752.
Suggested changes
- Synchronize those English and zh-Hans bundled-skill pages with this correction.
Automated hermes-sweeper review.
| @@ -343,7 +342,7 @@ The registry of record is `hermes_cli/commands.py` — every consumer | |||
|
|
|||
There was a problem hiding this comment.
Please synchronize this correction, plus the removed /skill references, to the published English and zh-Hans bundled-skill pages. Both currently still advertise /skill and /quit (/exit, /q) (website/docs/user-guide/skills/bundled/autonomous-ai-agents/autonomous-ai-agents-hermes-agent.md:307, :353, :850; localized mirror :296, :342, :752).
…rences Audited every reference against ground truth (COMMAND_REGISTRY, argparse --help output, TOOLSETS dict, provider profiles, DEFAULT_CONFIG) and corrected drift the restructure had carried over verbatim: - slash-commands.md: rebuilt from the live registry. Removes the phantom /skill command (never existed — skills load via /skills, hermes -s, or a skill's own /<name> command) and the wrong /q alias on /quit (/q belongs to /queue) — both reported by @liuhao1024 in #50608 and @gauravsaxena1997 in #50613. Adds ~25 real commands that were missing (/learn, /memory, /pet, /hatch, /bundles, /moa, /suggestions, /blueprint, /whoami, …) with correct aliases and CLI/GW scoping. - cli-reference.md: verified flags (-z/--oneshot, --tui/--cli, --safe-mode), real subcommand sets for config/setup/sessions/skills/gateway/mcp/profile/ auth/webhook/cron/skin/pets, added missing top-level commands (fallback, project, moa, logs, console, hooks, security, backup); condensed tables. - providers-and-models.md: rebuilt from the 36 shipped provider profiles with their actual env vars (was 21 rows, several stale); folds in the alias system and fallback-chain command. - configuration.md: toolset table matches TOOLSETS (adds coding, computer_use, video_gen, context_engine, project; drops nonexistent messaging/rl), config sections match DEFAULT_CONFIG, STT/TTS provider lists match the shipped set. - background-systems.md: curator verbs match the argparse surface. - troubleshooting/security-privacy/windows-quirks/contributor-guide: fixed dangling 'section above/below' references from the body split; moved the reset-permissions playbook to security-privacy.md (its routing home). Reported-by: liuhao1024 <sunsky.lau@gmail.com> Reported-by: gauravsaxena1997
…rences Audited every reference against ground truth (COMMAND_REGISTRY, argparse --help output, TOOLSETS dict, provider profiles, DEFAULT_CONFIG) and corrected drift the restructure had carried over verbatim: - slash-commands.md: rebuilt from the live registry. Removes the phantom /skill command (never existed — skills load via /skills, hermes -s, or a skill's own /<name> command) and the wrong /q alias on /quit (/q belongs to /queue) — both reported by @liuhao1024 in #50608 and @gauravsaxena1997 in #50613. Adds ~25 real commands that were missing (/learn, /memory, /pet, /hatch, /bundles, /moa, /suggestions, /blueprint, /whoami, …) with correct aliases and CLI/GW scoping. - cli-reference.md: verified flags (-z/--oneshot, --tui/--cli, --safe-mode), real subcommand sets for config/setup/sessions/skills/gateway/mcp/profile/ auth/webhook/cron/skin/pets, added missing top-level commands (fallback, project, moa, logs, console, hooks, security, backup); condensed tables. - providers-and-models.md: rebuilt from the 36 shipped provider profiles with their actual env vars (was 21 rows, several stale); folds in the alias system and fallback-chain command. - configuration.md: toolset table matches TOOLSETS (adds coding, computer_use, video_gen, context_engine, project; drops nonexistent messaging/rl), config sections match DEFAULT_CONFIG, STT/TTS provider lists match the shipped set. - background-systems.md: curator verbs match the argparse surface. - troubleshooting/security-privacy/windows-quirks/contributor-guide: fixed dangling 'section above/below' references from the body split; moved the reset-permissions playbook to security-privacy.md (its routing home). Reported-by: liuhao1024 <sunsky.lau@gmail.com> Reported-by: gauravsaxena1997
…rences Audited every reference against ground truth (COMMAND_REGISTRY, argparse --help output, TOOLSETS dict, provider profiles, DEFAULT_CONFIG) and corrected drift the restructure had carried over verbatim: - slash-commands.md: rebuilt from the live registry. Removes the phantom /skill command (never existed — skills load via /skills, hermes -s, or a skill's own /<name> command) and the wrong /q alias on /quit (/q belongs to /queue) — both reported by @liuhao1024 in #50608 and @gauravsaxena1997 in #50613. Adds ~25 real commands that were missing (/learn, /memory, /pet, /hatch, /bundles, /moa, /suggestions, /blueprint, /whoami, …) with correct aliases and CLI/GW scoping. - cli-reference.md: verified flags (-z/--oneshot, --tui/--cli, --safe-mode), real subcommand sets for config/setup/sessions/skills/gateway/mcp/profile/ auth/webhook/cron/skin/pets, added missing top-level commands (fallback, project, moa, logs, console, hooks, security, backup); condensed tables. - providers-and-models.md: rebuilt from the 36 shipped provider profiles with their actual env vars (was 21 rows, several stale); folds in the alias system and fallback-chain command. - configuration.md: toolset table matches TOOLSETS (adds coding, computer_use, video_gen, context_engine, project; drops nonexistent messaging/rl), config sections match DEFAULT_CONFIG, STT/TTS provider lists match the shipped set. - background-systems.md: curator verbs match the argparse surface. - troubleshooting/security-privacy/windows-quirks/contributor-guide: fixed dangling 'section above/below' references from the body split; moved the reset-permissions playbook to security-privacy.md (its routing home). Reported-by: liuhao1024 <sunsky.lau@gmail.com> Reported-by: gauravsaxena1997
|
Closing with credit — your fix landed via #70511 (merged as 46c7a40). You were the earliest to catch both errors (phantom |
…rences Audited every reference against ground truth (COMMAND_REGISTRY, argparse --help output, TOOLSETS dict, provider profiles, DEFAULT_CONFIG) and corrected drift the restructure had carried over verbatim: - slash-commands.md: rebuilt from the live registry. Removes the phantom /skill command (never existed — skills load via /skills, hermes -s, or a skill's own /<name> command) and the wrong /q alias on /quit (/q belongs to /queue) — both reported by @liuhao1024 in NousResearch#50608 and @gauravsaxena1997 in NousResearch#50613. Adds ~25 real commands that were missing (/learn, /memory, /pet, /hatch, /bundles, /moa, /suggestions, /blueprint, /whoami, …) with correct aliases and CLI/GW scoping. - cli-reference.md: verified flags (-z/--oneshot, --tui/--cli, --safe-mode), real subcommand sets for config/setup/sessions/skills/gateway/mcp/profile/ auth/webhook/cron/skin/pets, added missing top-level commands (fallback, project, moa, logs, console, hooks, security, backup); condensed tables. - providers-and-models.md: rebuilt from the 36 shipped provider profiles with their actual env vars (was 21 rows, several stale); folds in the alias system and fallback-chain command. - configuration.md: toolset table matches TOOLSETS (adds coding, computer_use, video_gen, context_engine, project; drops nonexistent messaging/rl), config sections match DEFAULT_CONFIG, STT/TTS provider lists match the shipped set. - background-systems.md: curator verbs match the argparse surface. - troubleshooting/security-privacy/windows-quirks/contributor-guide: fixed dangling 'section above/below' references from the body split; moved the reset-permissions playbook to security-privacy.md (its routing home). Reported-by: liuhao1024 <sunsky.lau@gmail.com> Reported-by: gauravsaxena1997
…rences Audited every reference against ground truth (COMMAND_REGISTRY, argparse --help output, TOOLSETS dict, provider profiles, DEFAULT_CONFIG) and corrected drift the restructure had carried over verbatim: - slash-commands.md: rebuilt from the live registry. Removes the phantom /skill command (never existed — skills load via /skills, hermes -s, or a skill's own /<name> command) and the wrong /q alias on /quit (/q belongs to /queue) — both reported by @liuhao1024 in NousResearch#50608 and @gauravsaxena1997 in NousResearch#50613. Adds ~25 real commands that were missing (/learn, /memory, /pet, /hatch, /bundles, /moa, /suggestions, /blueprint, /whoami, …) with correct aliases and CLI/GW scoping. - cli-reference.md: verified flags (-z/--oneshot, --tui/--cli, --safe-mode), real subcommand sets for config/setup/sessions/skills/gateway/mcp/profile/ auth/webhook/cron/skin/pets, added missing top-level commands (fallback, project, moa, logs, console, hooks, security, backup); condensed tables. - providers-and-models.md: rebuilt from the 36 shipped provider profiles with their actual env vars (was 21 rows, several stale); folds in the alias system and fallback-chain command. - configuration.md: toolset table matches TOOLSETS (adds coding, computer_use, video_gen, context_engine, project; drops nonexistent messaging/rl), config sections match DEFAULT_CONFIG, STT/TTS provider lists match the shipped set. - background-systems.md: curator verbs match the argparse surface. - troubleshooting/security-privacy/windows-quirks/contributor-guide: fixed dangling 'section above/below' references from the body split; moved the reset-permissions playbook to security-privacy.md (its routing home). Reported-by: liuhao1024 <sunsky.lau@gmail.com> Reported-by: gauravsaxena1997
…rences Audited every reference against ground truth (COMMAND_REGISTRY, argparse --help output, TOOLSETS dict, provider profiles, DEFAULT_CONFIG) and corrected drift the restructure had carried over verbatim: - slash-commands.md: rebuilt from the live registry. Removes the phantom /skill command (never existed — skills load via /skills, hermes -s, or a skill's own /<name> command) and the wrong /q alias on /quit (/q belongs to /queue) — both reported by @liuhao1024 in NousResearch#50608 and @gauravsaxena1997 in NousResearch#50613. Adds ~25 real commands that were missing (/learn, /memory, /pet, /hatch, /bundles, /moa, /suggestions, /blueprint, /whoami, …) with correct aliases and CLI/GW scoping. - cli-reference.md: verified flags (-z/--oneshot, --tui/--cli, --safe-mode), real subcommand sets for config/setup/sessions/skills/gateway/mcp/profile/ auth/webhook/cron/skin/pets, added missing top-level commands (fallback, project, moa, logs, console, hooks, security, backup); condensed tables. - providers-and-models.md: rebuilt from the 36 shipped provider profiles with their actual env vars (was 21 rows, several stale); folds in the alias system and fallback-chain command. - configuration.md: toolset table matches TOOLSETS (adds coding, computer_use, video_gen, context_engine, project; drops nonexistent messaging/rl), config sections match DEFAULT_CONFIG, STT/TTS provider lists match the shipped set. - background-systems.md: curator verbs match the argparse surface. - troubleshooting/security-privacy/windows-quirks/contributor-guide: fixed dangling 'section above/below' references from the body split; moved the reset-permissions playbook to security-privacy.md (its routing home). Reported-by: liuhao1024 <sunsky.lau@gmail.com> Reported-by: gauravsaxena1997
What does this PR do?
Removes two phantom slash commands from the hermes-agent SKILL.md that don't exist in the command registry, preventing agents from suggesting non-existent commands to users.
Related Issue
Fixes #50605
Type of Change
Changes Made
skills/autonomous-ai-agents/hermes-agent/SKILL.md: Remove/skill <name>line (no such command exists; only/skillsis registered)skills/autonomous-ai-agents/hermes-agent/SKILL.md: Fix/quitalias list — remove/q(it is an alias for/queue, not/quit)skills/autonomous-ai-agents/hermes-agent/SKILL.md: Update troubleshooting section to referencehermes -s nameinstead of nonexistent/skillHow to Test
skills/autonomous-ai-agents/hermes-agent/SKILL.md/skill <name>no longer appears in the Tools & Skills section (line ~300)/quit (/exit)in the Exit section no longer includes/qhermes -s nameonlyhermes_cli/commands.py: confirm/queuehas aliasq(line 107) and/quithas aliasexitonly (line 238)Checklist
Code
fix(scope):,feat(scope):, etc.)pytest tests/ -qand all tests passDocumentation & Housekeeping
docs/, docstrings) — or N/Acli-config.yaml.exampleif I added/changed config keys — or N/ACONTRIBUTING.mdorAGENTS.mdif I changed architecture or workflows — or N/ACode Intelligence
hermes_cli/commands.pylines 106-108 (/queuealiasq), 237-238 (/quitaliasexit)hermes_cli/commands.pydefines all valid slash commands